-
Notifications
You must be signed in to change notification settings - Fork 28
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update docs for Package Manager Postgres.UsageDataPassword
bug
#561
base: main
Are you sure you want to change the base?
Conversation
@@ -1,5 +1,9 @@ | |||
# Changelog | |||
|
|||
## 0.5.35 | |||
|
|||
- Update PostgreSQL configuration documentation to temporarily work around bug with `Postgres.UsageDataPassword` in Package Manager 2024.08.2. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Where is this update?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh what, wow, the automated helm-docs rebuild overwrote my readme change from 0995998
I might have to recreate this PR 🤔. I messed up originally by not bumping the chart version when making the readme change, and that might have messed things up?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I totally missed that there was a readme template file. Should look right now.
4b2c2bd
to
d170d9e
Compare
Postgres.UsageDataPassword
is supposed to default toPostgres.Password
, but a bug in recent PPM versions prevents this from working. So as a temporary workaround, thePostgres.UsageDataPassword
setting must be explicitly set toPostgres.Password
when using Postgres with usage data enabled.This will be fixed in the next Package Manager release. The config can be removed after the next release, but it also won't cause any issues if you keep it around for the next release.